Re sub/august updatefig #30

mfs4rd · 2025-11-03T23:10:01Z

Done

Update inputs (and documentation) of inside-outside_classification.py to default to the example movie used in paper and output to default figure folder
Updated inside-outside code to use the new z-thresholding approach

ToDo

need help updating pdm.lock file

pgarrison · 2025-11-07T19:55:33Z

EMT_data_analysis/analysis_scripts/Analysis_tools.py

    ]]

    df_io = io.load_inside_outside_classification()
+    df_io = df_io[df_io['Z']<27]


I tried to run this script and got an error on this line?

Traceback (most recent call last): File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3805, in get_loc return self._engine.get_loc(casted_key) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "index.pyx", line 167, in pandas._libs.index.IndexEngine.get_loc File "index.pyx", line 196, in pandas._libs.index.IndexEngine.get_loc File "pandas/_libs/hashtable_class_helper.pxi", line 7081, in pandas._libs.hashtable.PyObjectHashTable.get_item File "pandas/_libs/hashtable_class_helper.pxi", line 7089, in pandas._libs.hashtable.PyObjectHashTable.get_item KeyError: 'Z' The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/analysis_scripts/Analysis_tools.py", line 1558, in <module> run_all_analyses() File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/analysis_scripts/Analysis_tools.py", line 41, in run_all_analyses plot_inside_outside_migration_timing(df, FIGS_DIR, OUT_TYPE) File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/analysis_scripts/Analysis_tools.py", line 952, in plot_inside_outside_migration_timing dfio_merge = load_io_data(df) ^^^^^^^^^^^^^^^^ File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/analysis_scripts/Analysis_tools.py", line 91, in load_io_data df_io = df_io[df_io['Z']<27] ~~~~~^^^^^ File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/frame.py", line 4102, in __getitem__ indexer = self.columns.get_loc(key) ^^^^^^^^^^^^^^^^^^^^^^^^^ File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3812, in get_loc raise KeyError(key) from err KeyError: 'Z'

This is because the current manifest on quilt doesn't have the coordinates of nuclei centroids. For testing I temporarily changed the manifest to point to the copy currently on VAST but it needs to be uploaded
/allen/aics/users/filip.sluzewski/Public_Repos/emt-data-analysis/resubmission_scripts/nuclei_localization/mesh_features-resegmentation.csv

pgarrison · 2025-11-07T20:00:00Z

README.md

-This will generate the plots in the manuscript and store them in `results/figures` folder. The manifests used as inputs in this workflow are automatically downloaded from [AWS](https://open.quiltdata.com/b/allencell/tree/aics/emt_timelapse_dataset/manifests/) by default. The user can opt to also use local version of these manifests if they produced locally by running the scripts `Feature_extraction.py`, `Metric_computation.py` and `Nuclei_localization.py`. To use local version of the manifests, please set `load_from_aws=False` everywhere in the script `Analysis_plots.py`.
+This will generate the plots in the manuscript and store them in `results/figures` folder. The manifests used as inputs in this workflow are automatically downloaded from [AWS](https://open.quiltdata.com/b/allencell/tree/aics/emt_timelapse_dataset/manifests/) by default. 
+
+## 5 - [Optional] 3D Example Rendering


Suggested change

## 5 - [Optional] 3D Example Rendering

## 5 - 3D Example Rendering

I was under the impression that all the steps are optional? Do the other steps depend on the results of previous steps?

Technically no, as all of the code pulls from the quilt dataset, but if the user were to process their own data each step would be dependent on the previous

Gotcha. If you want to make those relationships between steps explicit, I'd recommend writing more of an introduction at the top of the README.

Our goal for reproducibility for this repo is just that people can run our code on our data and produce the figures in the paper: if we want to try to support users running on their own data, there's a lot more work we have to do.

pgarrison · 2025-11-07T20:30:05Z

README.md

+
+## 5 - [Optional] 3D Example Rendering
+
+The functions in `EMT_data_analysis/figure_generation` can be used to generate 3D renderings shown in the paper. Functions have only been tested on Ubuntu 18.04/22.04


Suggested change

The functions in `EMT_data_analysis/figure_generation` can be used to generate 3D renderings shown in the paper. Functions have only been tested on Ubuntu 18.04/22.04

The functions in `EMT_data_analysis/figure_generation` can be used to generate 3D renderings shown in the paper.

At the top of the readme we already said our code was tested on 18.04. If some of the code doesn't work on 18.04 and needs 22.04, that's a different thing. The top of the readme specifies 18.04.2 though, and our machines have upgraded to 18.04.4 and 18.04.6, so we should update that to be accurate to how we are testing.

The code will work on both 18.04 and 22.04 since the resubmission data was processed on the A100 machines. I don't think we need to be so specific as listing a specific sub-version of Ubuntu though

Unless we test all the code in this repo on 22.04, I think it's easier for users to understand "all the EMT_data_analysis code was tested on 18.04" than "all the EMT_data_analysis code was tested on 18.04, and also some of the code was tested on 22.04, too."

pgarrison · 2025-11-07T20:31:23Z

EMT_data_analysis/figure_generation/inside-outside_classification.py

    df_meta = df_meta[df_meta['Data ID'] == data_id]
    df = io.load_inside_outside_classification()
    df = df[df['Data ID'] == data_id]
+    df = df[df['Z']<27]


When I try to run this script it hits an error here.

$ pdm run EMT_data_analysis/figure_generation/inside-outside_classification.py Total number of movies in the dataset: 3491 Traceback (most recent call last): File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3805, in get_loc return self._engine.get_loc(casted_key) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "index.pyx", line 167, in pandas._libs.index.IndexEngine.get_loc File "index.pyx", line 196, in pandas._libs.index.IndexEngine.get_loc File "pandas/_libs/hashtable_class_helper.pxi", line 7081, in pandas._libs.hashtable.PyObjectHashTable.get_item File "pandas/_libs/hashtable_class_helper.pxi", line 7089, in pandas._libs.hashtable.PyObjectHashTable.get_item KeyError: 'Z' The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/figure_generation/inside-outside_classification.py", line 165, in <module> main(args.data_id, args.output) File "/home/philip.garrison/workspace/aics/EMT_data_analysis/EMT_data_analysis/figure_generation/inside-outside_classification.py", line 50, in main df = df[df['Z']<27] ~~^^^^^ File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/frame.py", line 4102, in __getitem__ indexer = self.columns.get_loc(key) ^^^^^^^^^^^^^^^^^^^^^^^^^ File "/home/philip.garrison/workspace/aics/EMT_data_analysis/.venv/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3812, in get_loc raise KeyError(key) from err KeyError: 'Z'

Same as the previous similar error, the quilt manifest needs to be updated to include this column

pgarrison · 2025-11-07T20:32:25Z

README.md

+```bash
+python colony_mask.py --data_id [Optional] --output_directory [Optional]
+```
+If no input arguments are provided, the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/3D_all_cells_mask`.


Suggested change

If no input arguments are provided, the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/3D_all_cells_mask`.

If no input arguments are provided (i.e., `python EMT_data_analysis/figure_generation/colony_mask.py`), the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/3D_all_cells_mask`.

input argument means --data_id or --output_directory

Yes, my suggestion was to provide clarity for users who might not understand that. For example, I could imagine someone leaving out the "[Optional]" pieces and running python colony_mask.py --data_id --output_directory.

pgarrison · 2025-11-07T20:32:41Z

README.md

+```bash
+python inside-outside_classification.py --data_id [Optional] --output_directory [Optional]
+```
+If no input arguments are provided, the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/Inside-Outside/mesh-figures`.


Suggested change

If no input arguments are provided, the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/Inside-Outside/mesh-figures`.

If no input arguments are provided (i.e., `python EMT_data_analysis/figure_generation/inside-outside_classification.py`), the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/Inside-Outside/mesh-figures`.

input argument means --data_id or --output_directory

README.md

Co-authored-by: Philip Garrison <[email protected]>

README.md

mfs4rd · 2025-11-10T21:00:40Z

@pgarrison I've pushed changes that should fix issues you've raised

mfs4rd added 2 commits November 3, 2025 15:02

inside-outside_classification default inputs updated

a4c3b4b

documentation update

cbe3b98

mfs4rd requested review from pgarrison and smishra3 November 3, 2025 23:10

mfs4rd added 6 commits November 4, 2025 12:39

updates to mesh-related dependencies for python 3.11

2ffc720

typo fix in colony_mask.pyw

fa90639

ACM mesh figure code and dependency update

c341338

Merge branch 'reSub-v1' into reSub/August_updatefig

07ae553

default colony mask output

73345f3

README documentation for figure_generation code

d637859

pgarrison requested changes Nov 7, 2025

View reviewed changes

pgarrison mentioned this pull request Nov 7, 2025

User documentation for colony_mask_features scripts #29

Open

pgarrison reviewed Nov 7, 2025

View reviewed changes

README.md Outdated Show resolved Hide resolved

pgarrison reviewed Nov 7, 2025

View reviewed changes

README.md Outdated Show resolved Hide resolved

mfs4rd and others added 2 commits November 7, 2025 12:56

Update README.md

affd91f

Co-authored-by: Philip Garrison <[email protected]>

Update README.md

44e74e7

Co-authored-by: Philip Garrison <[email protected]>

pgarrison reviewed Nov 7, 2025

View reviewed changes

README.md Show resolved Hide resolved

pgarrison and others added 6 commits November 7, 2025 14:01

Update pdm.lock with latest changes from pyproject.toml

40d5e90

remove pymeshfix import

7a5eae7

correct Data ID in nuclei localization

9702ca4

correcting camera alignment column name

af2f2d2

fix column name in feature extraction

f802b64

remove absolute path

7c8cf7d

README instructions assume user is in root folder of repo

cbe8372

mfs4rd requested a review from pgarrison November 10, 2025 22:22

	## 5 - [Optional] 3D Example Rendering
	## 5 - 3D Example Rendering


		## 5 - [Optional] 3D Example Rendering

		The functions in `EMT_data_analysis/figure_generation` can be used to generate 3D renderings shown in the paper. Functions have only been tested on Ubuntu 18.04/22.04

	If no input arguments are provided, the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/3D_all_cells_mask`.
	If no input arguments are provided (i.e., `python EMT_data_analysis/figure_generation/colony_mask.py`), the code will default to the data shown in the paper and output results to `EMT_data_analysis/results/3D_all_cells_mask`.

Re sub/august updatefig #30

Are you sure you want to change the base?

Re sub/august updatefig #30

Uh oh!

Conversation

mfs4rd commented Nov 3, 2025

Done

ToDo

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pgarrison Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mfs4rd commented Nov 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

pgarrison Nov 7, 2025 •

edited

Loading